Skip to content
This repository has been archived by the owner on Mar 2, 2023. It is now read-only.

Support PerRequestLifetimeManager #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Support PerRequestLifetimeManager #20

wants to merge 1 commit into from

Conversation

ctthang
Copy link

@ctthang ctthang commented Jun 28, 2021

No description provided.

@dnfadmin
Copy link

dnfadmin commented Jun 28, 2021

CLA assistant check
All CLA requirements met.

@codecov-commenter
Copy link

Codecov Report

Merging #20 (6a79f5f) into master (b684c2a) will decrease coverage by 36.52%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #20       +/-   ##
===========================================
- Coverage   81.39%   44.87%   -36.53%     
===========================================
  Files           2        4        +2     
  Lines          43       78       +35     
  Branches        2        9        +7     
===========================================
  Hits           35       35               
- Misses          6       41       +35     
  Partials        2        2               
Impacted Files Coverage Δ
src/PerRequestLifetimeManager.cs 0.00% <0.00%> (ø)
src/UnityPerRequestHttpModule.cs 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b684c2a...6a79f5f. Read the comment docs.

@ENikS
Copy link
Contributor

ENikS commented Jun 28, 2021

What are we fixing here?

@ctthang
Copy link
Author

ctthang commented Jun 30, 2021

What are we fixing here?

I need the PerRequestLifetimeManager in one of my projects so I move the implementation from the "aspnet-mvc" to here.
The reason for the need is that I want to manage my entity framework's dbcontext better for my REST API project.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants